Skip to content

Default protocol to UDP in the Log4j 1 SyslogAppender builder - #4247

Merged
ppkarwasz merged 2 commits into
apache:2.xfrom
DragonFSKY:fix-4238-syslog-default-udp
Sep 18, 2026
Merged

ppkarwasz merged 2 commits into
apache:2.xfrom
DragonFSKY:fix-4238-syslog-default-udp

Conversation

@DragonFSKY

@DragonFSKY DragonFSKY commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Fixes #4238.

Defaults the Log4j 1 SyslogAppender bridge protocol to UDP for XML and properties configurations, matching Log4j 1 behavior. Explicit protocol values remain unchanged.

Also documents protocol as a log4j-1.2-api extension and why stream transports are not recommended with Log4j1SyslogLayout.

Verification:

  • ./mvnw verify

@DragonFSKY
DragonFSKY force-pushed the fix-4238-syslog-default-udp branch from 9796eb6 to 80d92b3 Compare August 26, 2026 04:57
@vy vy added the vy label Sep 15, 2026

@ppkarwasz ppkarwasz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DragonFSKY,

LGTM! 💯 Thank you for your contribution.

@ppkarwasz ppkarwasz added the port-done No further port needed: all required ports are merged, or none was needed label Sep 18, 2026
@ppkarwasz
ppkarwasz enabled auto-merge (squash) September 18, 2026 06:48
@ppkarwasz
ppkarwasz merged commit e0c4d32 into apache:2.x Sep 18, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Log4j pull request tracker Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

port-done No further port needed: all required ports are merged, or none was needed vy

Projects

Development

Successfully merging this pull request may close these issues.

Default protocol to UDP in the Log4j 1 SyslogAppender builder

3 participants